Skip to content

[CORRUPTED] Synthetic Benchmark PR #26694 - refactor: Remove trpc/react dependency from @calcom/atoms #722

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
tomerqodo wants to merge 8 commits into
base: base_pr_26694_20260121_5561
Choose a base branch
from
Open

[CORRUPTED] Synthetic Benchmark PR #26694 - refactor: Remove trpc/react dependency from @calcom/atoms #722

tomerqodo wants to merge 8 commits into from

Conversation

@tomerqodo
Copy link

@tomerqodo tomerqodo commented Jan 21, 2026

Benchmark PR calcom#26694

Type: Corrupted (contains bugs)

Original PR Title: refactor: Remove trpc/react dependency from @calcom/atoms
Original PR Description: ## What does this PR do?

Decouples trpc from platform atoms by moving the useTeamMembersWithSegment hook into the web app and removing trpc references in atoms. This reduces cross-package coupling with no behavior changes.

  • Refactors
    • Moved useTeamMembersWithSegment to apps/web/modules/event-types/hooks.
    • Exported UseTeamMembersWithSegmentProps and useProcessTeamMembersData from atoms.
    • Removed trpc usage from atoms; platform hook stays as useTeamMembersWithSegmentPlatform.
    • Updated EditWeightsForAllTeamMembers to use the new web hook and corrected platform import path.
    • Replaced TRPC types in AvailabilitySettings with repository types and removed trpc/react from atoms tsconfig.

Impact

Screenshot 2026-01-11 at 7 13 58 PM

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Please use the latest Vercel preview and test please 🙏.

Original PR URL: calcom#26694

@github-actions
Copy link

github-actions bot commented Jan 21, 2026

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

No release type found in pull request title "[CORRUPTED] Synthetic Benchmark PR #26694 - refactor: Remove `trpc/react` dependency from @calcom/atoms". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

This was referenced Jan 21, 2026
Open
Open
Open
Open
Open
Open
Open
This was referenced Jan 26, 2026
Open
Open
@github-actions
Copy link

github-actions bot commented Jan 29, 2026

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.

@github-actions github-actions bot added the Stale label Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tomerqodo @hbjORbj